Improve code coverage with unit and integration tests by Copilot · Pull Request #32 · demaconsulting/SonarMark

Copilot · 2026-01-21T16:03:57Z

Code coverage was at 54.88%. Added 26 tests (12 unit tests + 14 integration tests) targeting uncovered paths in core classes.

Coverage Improvements

Overall: 54.88% → 59.11%
Context: 96.21% → 100%
SonarHotSpot: 85.71% → 100%
SonarIssue: 87.5% → 100%
Program: 40.47% → 58.33%

Changes

New test files:
- SonarHotSpotTests.cs - Record property validation and null handling
- SonarIssueTests.cs - Record property validation across all severity levels
- Runner.cs - Helper class for running SonarMark through dotnet for integration testing
- IntegrationTests.cs - 14 integration tests exercising the full application via command-line
Enhanced existing tests:
- ProgramTests.cs - Error handling for missing required parameters (--server, --project-key)
- ContextTests.cs - Invalid log file path exception handling

Integration Tests

The integration tests exercise the complete application by running it through dotnet exec, covering:

Version, help, and validate flags
Missing and invalid parameter error handling
Silent flag behavior
All command-line parameter acceptance (token, branch, enforce, report, report-depth)

All tests follow existing MSTest v4 conventions with ClassName_MethodUnderTest_Scenario_ExpectedBehavior naming.

Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature]: Improve Code Coverage</issue_title>
<issue_description>### Problem Statement

Code Coverage statistics need to be improved.

Proposed Solution

Add more tests to bring the code-coverage numbers up.

Alternatives Considered

No response

Additional Context

No response

Priority

Medium - Would improve workflow

Contribution

I would be willing to submit a PR for this feature

Code of Conduct

I agree to follow this project's Code of Conduct</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes [Feature]: Improve Code Coverage #31

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Malcolmnixon · 2026-01-21T16:15:24Z

@copilot The coverage numbers are still quite low. Consider enhancing the test project with:

A Runner class like (https://github.com/demaconsulting/VHDLTest/blob/main/test/DEMAConsulting.VHDLTest.Tests/Runner.cs)
Integration tests like (https://github.com/demaconsulting/VHDLTest/blob/main/test/DEMAConsulting.VHDLTest.Tests/ValidationTests.cs) which run the SonarMark program (through dotnet and the DLL) passing in program arguments to test different functionality.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Copilot · 2026-01-21T16:24:32Z

@copilot The coverage numbers are still quite low. Consider enhancing the test project with:

A Runner class like (https://github.com/demaconsulting/VHDLTest/blob/main/test/DEMAConsulting.VHDLTest.T...

Added Runner class and 14 integration tests following the VHDLTest pattern. Code coverage improved from 54.88% to 59.11%. All 72 tests pass. Changes in commit 4329472.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

* Initial plan * Apply TemplateDotNetTool consistency improvements - Add end_of_line=lf to .editorconfig for consistent line endings - Add MD025 rule to .markdownlint-cli2.jsonc and remove inline comments - Convert README badges to reference-style links matching template pattern - Add comprehensive XML documentation to Context.cs ArgumentParser class - Improve catch block documentation with expected exception types - Reorganize test project .csproj with section comments for clarity - Clean up workflow YAML comments to match template style All changes maintain functionality while improving consistency with the TemplateDotNetTool template standards. * Fix requirements test coverage linkage - Updated platform requirements (PLT-001 through PLT-005) to use correct test source patterns matching CI workflow naming - Changed from generic prefixes (windows@, ubuntu@, dotnet8.x@) to actual CI file patterns (windows-latest@, integration-test-*-dotnet*.x@) - Ensures requirements can be satisfied when full CI pipeline runs - Enhanced VAL-003 and VAL-004 test coverage by adding self-validation tests - These tests exercise both TRX and JUnit format support through the validation functionality - Provides comprehensive validation of format support requirements - All 24 locally-testable requirements are now satisfied - 5 platform-specific requirements (PLT-*) correctly remain unsatisfied locally (requires CI matrix) - Verified with 'dotnet reqstream --enforce' * fix: Exclude node_modules from yamllint checks The yamllint configuration was linting node_modules directory, causing false positives from third-party dependencies. This change adds an ignore section to exclude node_modules and .git directories from yamllint checks, focusing on project files only. * docs: Fix documentation formatting and add missing scope section Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Enhance XML documentation and literate programming style Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Improve test naming and fix cspell dictionary Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * docs: Restructure user guide heading hierarchy for PDF compilation Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * fix: Update requirements.yaml header comment and revert test source filter changes Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Synchronize with TemplateDotNetTool template PRs #29, #31, #32, #35 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Initial plan

e12e140

Copilot AI assigned Copilot and Malcolmnixon Jan 21, 2026

Copilot started work on behalf of Malcolmnixon January 21, 2026 16:04 View session

Add tests to improve code coverage - added 12 new tests

c8753c0

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Malcolmnixon approved these changes Jan 21, 2026

View reviewed changes

Copilot AI changed the title ~~[WIP] Add tests to improve code coverage~~ Improve code coverage with targeted unit tests Jan 21, 2026

Copilot AI requested a review from Malcolmnixon January 21, 2026 16:15

Copilot finished work on behalf of Malcolmnixon January 21, 2026 16:15

Copilot started work on behalf of Malcolmnixon January 21, 2026 16:16 View session

Copilot AI and others added 2 commits January 21, 2026 16:20

Add Runner class and integration tests for improved coverage

d1ee3af

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Fix Runner to use async stream reading and proper disposal

4329472

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Copilot AI changed the title ~~Improve code coverage with targeted unit tests~~ Improve code coverage with unit and integration tests Jan 21, 2026

Copilot finished work on behalf of Malcolmnixon January 21, 2026 16:25

Malcolmnixon approved these changes Jan 21, 2026

View reviewed changes

Malcolmnixon marked this pull request as ready for review January 21, 2026 16:45

Malcolmnixon merged commit eae7e37 into main Jan 21, 2026
9 checks passed

Malcolmnixon deleted the copilot/improve-code-coverage-tests branch January 21, 2026 16:45

Copilot AI added a commit that referenced this pull request Feb 20, 2026

Synchronize with TemplateDotNetTool template PRs #29, #31, #32, #35

eaabff6

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Copilot AI mentioned this pull request Feb 20, 2026

Pre-release quality sweep: all agents and template synchronization #89

Merged

22 tasks

Copilot AI mentioned this pull request Mar 6, 2026

Bring in recent changes from TemplateDotNetTool (PRs #53, #54, #55) #100

Merged

23 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improve code coverage with unit and integration tests#32

Improve code coverage with unit and integration tests#32
Malcolmnixon merged 4 commits intomainfrom
copilot/improve-code-coverage-tests

Copilot AI commented Jan 21, 2026 •

edited

Loading

Uh oh!

Malcolmnixon commented Jan 21, 2026

Uh oh!

Copilot AI commented Jan 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Copilot AI commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Improvements

Changes

Integration Tests

Proposed Solution

Alternatives Considered

Additional Context

Priority

Contribution

Code of Conduct

Comments on the Issue (you are @copilot in this section)

Uh oh!

Malcolmnixon commented Jan 21, 2026

Uh oh!

Copilot AI commented Jan 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Jan 21, 2026 •

edited

Loading